Skip to content

Update "upgrading" guide for Symfony5 #12673

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 23, 2019
Merged

Conversation

ThomasGallet
Copy link
Contributor

@ThomasGallet ThomasGallet commented Nov 23, 2019

Come from #12633 issue

#SymfonyHackday

phansys and others added 2 commits November 22, 2019 19:04
…xtension (phansys)

This PR was merged into the 4.4 branch.

Discussion
----------

[Messenger] Update version constraint for Redis extension

Update version constraint for Redis extension in order to be consistent with the check performed in the component:

https://github.com/symfony/symfony/blob/625a4dbfdafcb8cea8ff90a62b9c24b28694938d/src/Symfony/Component/Messenger/Transport/RedisExt/Connection.php#L50-L52.

Commits
-------

edb2101 [Messenger] Update version constraint for Redis extension
@wouterj wouterj changed the base branch from 5.0 to 4.4 November 23, 2019 12:45
wouterj added a commit that referenced this pull request Nov 23, 2019
This PR was submitted for the 5.0 branch but it was merged into the 4.4 branch instead (closes #12673).

Discussion
----------

Update "upgrading" guide for Symfony5

Come from #12633 issue

#SymfonyHackday

Commits
-------

498dff9 Update "upgrading" guide for Symfony5
@wouterj wouterj merged commit 498dff9 into symfony:4.4 Nov 23, 2019
@wouterj
Copy link
Member

wouterj commented Nov 23, 2019

Hi! Thank you for working on this one.

Unfortunately, two people (with very similair names!) worked on the same topic: #12671. I've decided to merge this PR and add some improvements from #12671 in it afterwards. So the result on symfony.com will be very similair to your changes :)

I've also merged this in 4.4 (as it's talking about upgrading from 4.4 to 5.0, I think it makes sense to have it in 4.4). We'll take care of merging it into newer versions from here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants